Skip to content

Update dockerd.md#3173

Merged
vvoland merged 1 commit into
docker:masterfrom
rene-hermenau:patch-1
May 20, 2026
Merged

Update dockerd.md#3173
vvoland merged 1 commit into
docker:masterfrom
rene-hermenau:patch-1

Conversation

@rene-hermenau
Copy link
Copy Markdown

daemon.json does not exist on a clean install. The doc should reflect that.

- What I did

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

daemon.json does not exist on a clean install. The doc should reflect that.

Signed-off-by: René Hermenau <rene-hermenau@users.noreply.github.com>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Copy link
Copy Markdown
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

did a rebase and minor touch-up

@thaJeztah thaJeztah requested a review from vvoland May 14, 2026 22:35
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@rene-hermenau
Copy link
Copy Markdown
Author

Thanks @thaJeztah I totally forgot this 5 year old PR🤣

@thaJeztah
Copy link
Copy Markdown
Member

Heh, yeah, definitely the same; I was spending a bit of time going through old PRs. Not sure why this wasn't merged at the time, but .. not unlikely I looked at it from my phone and wanted to check it from my computer later and then forgot.

@vvoland vvoland modified the milestones: 29.5.1, 29.5.2 May 18, 2026
@vvoland
Copy link
Copy Markdown
Collaborator

vvoland commented May 20, 2026

#6997 should fix the failing milestone check

@thaJeztah
Copy link
Copy Markdown
Member

@vvoland yeah, didn't want to do a rebase just for that, so we can probably just ignore it? but I can't review this PR because I did the last rebase 😅

@thaJeztah
Copy link
Copy Markdown
Member

Oh! Sorry, thought You linked to the "VERSION" bump PR; nevermind 🙈

@vvoland
Copy link
Copy Markdown
Collaborator

vvoland commented May 20, 2026

We can't merge because the check is required 😅
Can you rebase please?

@thaJeztah
Copy link
Copy Markdown
Member

I don't think it's required;

Screenshot 2026-05-20 at 16 24 27

I think it's blocking because my review no longer counts;

Screenshot 2026-05-20 at 16 24 50

@vvoland
Copy link
Copy Markdown
Collaborator

vvoland commented May 20, 2026

Oh right!

@vvoland vvoland merged commit 79eb04c into docker:master May 20, 2026
92 of 96 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants